fix(dns): ignore volatile SOA serial on zone import; DNS validated live - #30
Merged
Conversation
The pcd_dns_zone import test compared the Designate-managed SOA serial, which Designate auto-increments out of band, so the value re-read on import differed from the last-applied state (ImportStateVerify mismatch). Ignore `serial` on the zone import step. Validated live against a Designate pool backed by BIND9: zone + recordset create -> ACTIVE, verify via the API, import, and destroy. This clears the last lab-blocked suite; every resource mapping to a configured PCD backend is now validated on the CE lab (see DECISIONS.md).
PF9-pushkar
added a commit
that referenced
this pull request
Jul 14, 2026
fix(dns): ignore volatile SOA serial on zone import; DNS validated live
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ran the Designate acceptance tests against the CE lab now that the DNS pool is backed by BIND9.
Fix
The
pcd_dns_zoneimport test compared the SOAserial, which Designate auto-increments out of band (the value jumped…6879→…6889between apply and the import re-read), causing anImportStateVerifymismatch. AddedImportStateVerifyIgnore: ["serial"]to the zone import step —serialis a legitimately volatile server-managed field.Validated live (2026-07-13)
TestAccDNSZoneAndRecordSet_basicpasses end to end:pcd_dns_zone+pcd_dns_recordsetcreate →ACTIVE, verified via the Designate API, imported, and destroyed. Pool backed by BIND9 on the hypervisor per the runbook's Phase E step 7.This clears the last lab-blocked suite. Every provider resource that maps to a configured PCD backend is now validated live on the CE lab.
Checks
go build,go vet,gofmt,golangci-lint(0 issues) — clean.